Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing all the references from Flash to OpenFL. #2832

Merged
merged 3 commits into from
Jun 8, 2023
Merged

Changing all the references from Flash to OpenFL. #2832

merged 3 commits into from
Jun 8, 2023

Conversation

MAJigsaw77
Copy link
Contributor

The title tells everything this commit do.

@Geokureli
Copy link
Member

ive been putting this off because it will likely cause merge conflicts with almost every other active PR, but let's try it. gimme some time, first

import openfl.geom.Matrix;
import openfl.geom.Point;
import openfl.ui.MouseCursor;
import openfl.ui.MouseCursorData;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like flash is failing here because there is no openfl.ui.MouseCursorData

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think openfl doesn't have support for that classes but we can find a workaround ig?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/HaxeFlixel/flixel/blob/dev/flixel/system/macros/FlxDefines.hx#L144
I think FLX_NATIVE_CURSOR is only used with flash, so lets just use flash.ui.MouseCursorData

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is that a question?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah

@Geokureli Geokureli merged commit 528620c into HaxeFlixel:dev Jun 8, 2023
@Geokureli
Copy link
Member

Thanks, hopefully this fixes most macro issues

Geokureli added a commit that referenced this pull request Jun 15, 2023
@Geokureli Geokureli added this to the 5.4.0 milestone Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants